Skip to content

Typescript function syntax#17

Open
timmyg wants to merge 5 commits intozacharytamas:masterfrom
timmyg:typescript-function-syntax
Open

Typescript function syntax#17
timmyg wants to merge 5 commits intozacharytamas:masterfrom
timmyg:typescript-function-syntax

Conversation

@timmyg
Copy link

@timmyg timmyg commented Dec 2, 2016

not sure if this is the best way to do this or even intended. I created a atom-jasmine-typescript atom package (my first one) but would prefer for this to be available in your package.

I am not sure how to make things configurable so I just did it this way.

@zacharytamas
Copy link
Owner

Hey Tim, yeah they make it pretty hard to make it configurable. This seems a sensible approach.

One thing I wonder about though is whether this causes a problem for Jasmine given that the this would be different. Some people persist test variables on this between their beforeEach, etc. and it functions. Does lexical this mess that up? I'm a bit out of practice with Jasmine these days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants